Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(surveys): Publish types in module #863

Merged
merged 1 commit into from
Oct 27, 2023
Merged

fix(surveys): Publish types in module #863

merged 1 commit into from
Oct 27, 2023

Conversation

neilkakkar
Copy link
Collaborator

Changes

Publish survey types alongside other types in the module. @xrdt noticed this wasn't working, which isn't great for API based surveys.
...

Confirmed this issue doesn't come back: #698

Tested that import { Survey } from "posthog-js" works

Checklist

@neilkakkar neilkakkar added the bump patch Bump patch version when this PR gets merged label Oct 27, 2023
@github-actions
Copy link

Size Change: +570 B (0%)

Total Size: 711 kB

Filename Size Change
dist/es.js 112 kB +245 B (0%)
dist/module.js 112 kB +325 B (0%)
ℹ️ View Unchanged
Filename Size
dist/array.full.js 170 kB
dist/array.js 112 kB
dist/exception-autocapture.js 12 kB
dist/recorder-v2.js 95.1 kB
dist/recorder.js 58.4 kB
dist/surveys.js 39.7 kB

compressed-size-action

@neilkakkar neilkakkar marked this pull request as ready for review October 27, 2023 10:47
@neilkakkar neilkakkar merged commit 991355f into master Oct 27, 2023
14 checks passed
@neilkakkar neilkakkar deleted the survey-types branch October 27, 2023 10:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bump patch Bump patch version when this PR gets merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants